Feature/세미나 출석 코드 입력 제한 + 최근의 세미나 조회 기능 구현 by shkisme · Pull Request #232 · KEEPER31337/Homepage-Back-R2

shkisme · 2023-07-21T14:43:27Z

🔥 Related Issue

close: #215
close: #216

📝 Description

세미나 출석 코드 입력 5회 초과 못하게 처리
- 홈페이지 출석과 마찬가지로 Redis에 값을 띄워서 처리했습니다.
마감된 세미나 중 가장 최근의 세미나 조회
- (~어제) 까지의 세미나 중 가장 현재와 가깝고 마감된 세미나를 조회합니다.
예정된 세미나 중 가장 최근의 세미나 2개 조회
- (오늘~) 의 세미나 중 가장 현재와 가까운 세미나 2개를 조회합니다.

⭐️ Review Request

기획서

세미나 도메인 어렵네요....!

gusah009

수고하셨습니다~ 👍👍

gusah009 · 2023-07-21T15:07:43Z

  @Transactional
-  public SeminarAttendanceResponse save(Long seminarId, Member member, SeminarAttendanceCodeRequest request){
+  public SeminarAttendanceResponse save(Long seminarId, Member member, SeminarAttendanceCodeRequest request) {
+    String key = "seminar:" + LocalDate.now() + "memberId:" + member.getId();


오오 redis key naming convention을 사용하셨군요..! 부끄럽게도 전 redis 만질 때 저걸 몰라서 key name을 막 지었습니다 ㅎㅎ...

nit)

Suggested change

String key = "seminar:" + LocalDate.now() + "memberId:" + member.getId();

String key = "seminar:" + LocalDate.now() + ":memberId:" + member.getId();

그러고 보니 야구게임, 로그인 쪽에서는 _ 로 키 이름이 구분지어져 있네요! 통일성 있게 _ 로 수정하겠습니다!

아 아니에용 @shkisme 님의 naming convention에 맞게 야구 게임쪽을 고치겠습니다~~

gusah009 · 2023-07-21T15:10:10Z

+
+
+    @Test
+    @DisplayName("출석코드 5회 초과 입력 시 출석이 불가능해야 한다.")


👍👍👍

gusah009 · 2023-07-21T15:11:18Z

        .seminarAttendanceStatus(seminar.getStatus())
        .build();
-    return SeminarAttendanceResponse.of(attendanceRepository.save(attendance), attemptNumber);
+    return SeminarAttendanceResponse.of(attendanceRepository.save(attendance));


출석 시도 횟수를 응답으로 내려주지 않도록 변경한 이유가 있을까요?

처음에는 시도 횟수를 출석 코드가 틀렸을 때 마다 반환하려고 했는데요! 출석을 성공했을 경우에만 출석 시도 횟수를 성공 응답으로 보내주게 되어서 (출석 코드를 틀렸을 경우에는 400 응답을 보내게 되니...) 필요가 없을 것 같아서 뺐습니다! 프론트에서 처리가 가능한 지 문의드렸고 확인중이라고 합니다!

이 부분 혹시 다르게 처리할 수 있는 방법이 있을까요? 🤔

hyuunnn

두둥등장...!

hyuunnn · 2023-07-21T16:08:35Z

+    int attemptNumber = redisUtil.increaseAndGetWithExpire(key, 60 * 60).intValue(); // 1시간 후 만료
+    if (attemptNumber > 5) {


주석을 static 변수로 표현하면 지울 수 있겠네요. (60 * 60)
5를 의미하는 static 변수도 만드는게 좋을까요..?

MAX_ATTEMPT_COUNT 같은걸로 해도 괜찮을 것 같네요~

@hyuunnn 오랜만입니다 ㅎㅎ

변수화하겠습니다~! 👍

hyuunnn · 2023-07-21T16:12:04Z

+    return seminarRepository.findRecentlyDoneSeminar(now)
+        .map(Seminar::getId)
+        .map(SeminarIdResponse::new)
+        .orElse(null);
+  }
+
+  public List<SeminarIdResponse> getRecentlyUpcomingSeminars() {
+    LocalDate now = LocalDate.now();
+    return seminarRepository.findRecentlyUpcomingSeminar(now)
+        .stream()
+        .map(Seminar::getId)
+        .map(SeminarIdResponse::new)
+        .toList();


SeminarIdResponse 클래스가 제가 만든거지만..
dto 역할을 하는, 데이터만 넘겨주는 껍데기인데
사용할 때마다 새로운 object를 new를 통해 생성하는게 과연 효율적인지에 대해 고민이 들더라구요..

자주 사용하시는 of처럼 static 메서드로 만들어서 싱글톤 패턴으로 만든다면..? 여기까지 ㅎㅎ..

new로 만드는게 좋을 것 같아요~ 요청마다 SemiarId는 다른 값을 가지게 될텐데 싱글톤으로 만들어버리면 동시성 문제 때문에 값이 섞일 수 있을 것 같네요~

저도 DTO, 싱글톤에 대해서 한번 더 생각해보게 되네요!

@gusah009 아 그러네요 중간에 값이 바뀔 수가 있네요.. 잘못 생각했습니다...
다른거 공부하다 보니까 바보 멍청이가 되었습니다...

절대 쿠버네티스를 해선 안돼

gusah009 · 2023-07-22T06:35:33Z

-    return attemptNumber;
  }

  private void validAttendanceCode(Seminar seminar, SeminarAttendanceCodeRequest request) {


이 부분 혹시 다르게 처리할 수 있는 방법이 있을까요? 🤔

여기서 throw로 던져줄 때 inputAttendanceCode 대신 attemptNumber를 던져주는 방법도 있을 것 같네용~

에러와 함께 시도횟수를 주는 방법도 있었네요! 이 방법으로 수정해보겠습니다!

- 세미나 출석은 정회원만 가능하게 구현하기 위함.

- 회원도 접근 가능하도록 - 자신의 출석 상태도 응답으로 반환하도록 (만약 출석 정보가 없을 시는 `출석 전` 상태를 반환)

- 출석 api의 HTTP 메서드를 @PatchMapping으로 변경 - 틀린 출석 코드 입력할 경우 에러 메시지와 함께 시도 횟수를 반환하도록 - 출석 정보를 찾을 수 없는 경우 에러 반환

- pathVariable로 회원 id를 받아 수정 가능하도록 - 추후 관리자용 api가 될 예정임을 TODO로 명시

shkisme · 2023-07-23T09:00:51Z

프론트에서 얼른 테스트해봐야 할 거 같아서 여기서 작업하고 바로 머지하겠습니다!

작업요약

예전 홈페이지 처럼 세미나 생성 시 모든 정회원들의 출석을 출석 전 상태로 저장하고, 출석 api 호출 시 출석 상태를 변경합니다.
세미나 조회 시 회원의 출석 상태도 응답으로 반환합니다.

뭔가 상벌점 도메인이랑 엮어서 생각하니 꽤나 복잡하고 아직 기획이 안 된 부분도 있어서 일단은 더 고민해보고 추후에 작업해야 할 것 같습니다!

출석 전 상태로 출석 하지 않고 남아있는 회원에게 벌점을 매겨야 할 텐데 어떻게 해야 할 지 고민
현재 구현 상으로는 세미나 생성 시에는 비회원이였으나, 세미나 출석 시에는 정회원이라면 문제가 발생하게 됨.
- 만약 세미나 일정 관리 페이지가 개발이 된다면, 문제가 있어서 꼭 수정해야함.

shkisme added 5 commits July 17, 2023 22:54

feat: 세미나 출석 코드 입력 5회 제한 코드 추가

d93fa18

style: 세미나 출석에 대한 테스트를 시행하는 클래스 명 SeminarAttendanceTest로 변경

02f0055

fix: attemtNumber를 성공 응답으로 반환하지 않게 수정

037e7dc

feat: 가장 최근에 마감된 세미나 조회 api 개발

7e5079b

feat: 예정된 세미나 중 가장 최근의 세미나 2개를 조회하는 api 구현

697f04b

shkisme added 🚀 feature 새로운 기능 구현 및 개선사항 ⭐️ backend 백엔드 코드 작성 labels Jul 21, 2023

shkisme requested review from JJimini, gusah009 and hyuunnn July 21, 2023 14:43

shkisme self-assigned this Jul 21, 2023

Merge branch 'develop' into feature/세미나-출석-추가기능-구현

27d60b4

gusah009 approved these changes Jul 21, 2023

View reviewed changes

hyuunnn approved these changes Jul 21, 2023

View reviewed changes

gusah009 approved these changes Jul 22, 2023

View reviewed changes

shkisme added 7 commits July 23, 2023 16:38

feat: MemberFindService.findAllRegular() 메서드 추가 (정회원 목록 조회)

52ae941

fix: 세미나 생성 시 모든 정회원의 출석을 출석 전 상태로 저장

3d34ae6

- 세미나 출석은 정회원만 가능하게 구현하기 위함.

fix: 세미나 조회 api 수정

249f978

- 회원도 접근 가능하도록 - 자신의 출석 상태도 응답으로 반환하도록 (만약 출석 정보가 없을 시는 `출석 전` 상태를 반환)

fix: 세미나 출석 api 수정

64f708f

- 출석 api의 HTTP 메서드를 @PatchMapping으로 변경 - 틀린 출석 코드 입력할 경우 에러 메시지와 함께 시도 횟수를 반환하도록 - 출석 정보를 찾을 수 없는 경우 에러 반환

fix: 세미나 출석 상태 변경 api 수정

b01c2de

- pathVariable로 회원 id를 받아 수정 가능하도록 - 추후 관리자용 api가 될 예정임을 TODO로 명시

test: 변경된 api에 대한 테스트 코드 수정 (추가한 테스트 x)

aee27ff

docs: api 문서에 누락된 Path Parameters 추가

9e91f6d

shkisme merged commit 5c8ff29 into develop Jul 23, 2023

shkisme deleted the feature/세미나-출석-추가기능-구현 branch July 23, 2023 09:00

shkisme mentioned this pull request Jul 23, 2023

[REFACTOR] 세미나 출석 api를 수정한다. #239

Open

	String key = "seminar:" + LocalDate.now() + "memberId:" + member.getId();
	String key = "seminar:" + LocalDate.now() + ":memberId:" + member.getId();



		@Test
		@DisplayName("출석코드 5회 초과 입력 시 출석이 불가능해야 한다.")

		int attemptNumber = redisUtil.increaseAndGetWithExpire(key, 60 * 60).intValue(); // 1시간 후 만료
		if (attemptNumber > 5) {

Uh oh!

Hidden character warning

Conversation

shkisme commented Jul 21, 2023

🔥 Related Issue

📝 Description

⭐️ Review Request

Uh oh!

gusah009 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

shkisme Jul 21, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hyuunnn left a comment

Choose a reason for hiding this comment

Uh oh!

hyuunnn Jul 21, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hyuunnn Jul 21, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hyuunnn Jul 22, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

shkisme commented Jul 23, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

작업요약

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

shkisme Jul 21, 2023 •

edited

Loading

hyuunnn Jul 21, 2023 •

edited

Loading

hyuunnn Jul 21, 2023 •

edited

Loading

hyuunnn Jul 22, 2023 •

edited

Loading

shkisme commented Jul 23, 2023 •

edited

Loading